Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: toolbar UI update (DHIS2-15167) #368

Merged
merged 44 commits into from
Aug 9, 2023

Conversation

HendrikThePendric
Copy link
Contributor

@HendrikThePendric HendrikThePendric commented May 24, 2023

Implements DHIS2-15167

Requires dhis2/analytics#1478


Key features

  1. Updated toolbar with hover menu menu

TODO

  • Update @dhis2/analytics once released.
  • Update tests for the new implementation

Screenshots

Screen Recording 2023-06-21 at 17 24 18

@dhis2-bot
Copy link
Contributor

dhis2-bot commented May 24, 2023

🚀 Deployed on https://pr-368--dhis2-line-listing.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify May 24, 2023 09:00 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify May 24, 2023 12:08 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify May 24, 2023 12:25 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify May 24, 2023 12:45 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 20, 2023 12:46 Inactive
@HendrikThePendric HendrikThePendric marked this pull request as ready for review June 20, 2023 13:28
@dhis2-bot dhis2-bot temporarily deployed to netlify June 20, 2023 13:34 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 20, 2023 13:54 Inactive
@cypress
Copy link

cypress bot commented Jun 20, 2023

19 failed and 4 flaky tests on run #1590 ↗︎

19 433 2 36 Flakiness 4

Details:

Merge 6b8f5e5 into 51ecd03...
Project: line-listing-app Commit: 5722955fdc ℹ️
Status: Failed Duration: 30:56 💡
Started: Jul 4, 2023 9:56 AM Ended: Jul 4, 2023 10:27 AM
Failed  fileMenu.cy.js • 2 failed tests • e2e-chrome-parallel-2.40

View Output Video

Test Artifacts
file menu > reflects "saved, valid: data" state Output Screenshots Video
file menu > reflects "dirty" state Output Screenshots Video
Failed  interpretations.cy.js • 1 failed test • e2e-chrome-parallel-2.40

View Output Video

Test Artifacts
interpretations > the interpretations panel can be toggled when clicking the button in the toolbar Output Screenshots Video
Failed  programDimensions.cy.js • 1 failed test • e2e-chrome-parallel-2.40

View Output Video

Test Artifacts
program dimensions > event > program can be selected and cleared Output Screenshots Video
Failed  table.cy.js • 1 failed test • e2e-chrome-parallel-2.40

View Output Video

Test Artifacts
table > dimensions display correct values in the visualization (>=2.39) Output Screenshots Video
Failed  timeDimensions.cy.js • 1 failed test • e2e-chrome-parallel-2.40

View Output Video

Test Artifacts
time dimensions > scheduledDate shows the correct title in layout and table header Output Screenshots Video

The first 5 failed specs are shown, see all 15 specs in Cypress Cloud.

Flakiness  options.cy.js • 1 flaky test • e2e-chrome-parallel-2.40

View Output Video

Test Artifacts
ou hierarchy > sets organisation unit hierarchy Output Screenshots
Flakiness  fileMenu.cy.js • 1 flaky test • e2e-chrome-parallel-2.39

View Output Video

Test Artifacts
file menu > reflects "saved, valid: save" state Output Screenshots Video
Flakiness  fileMenu.cy.js • 1 flaky test • e2e-chrome-parallel-2.38

View Output Video

Test Artifacts
file menu > reflects "saved, valid: save" state Output Screenshots Video
Flakiness  conditions/dateConditions.cy.js • 1 flaky test • e2e-chrome-parallel-2.38

View Output Video

Test Artifacts
date conditions (Date) > before Output Screenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@dhis2-bot dhis2-bot temporarily deployed to netlify June 21, 2023 11:20 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 21, 2023 13:57 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 21, 2023 14:12 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 21, 2023 14:29 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 21, 2023 15:15 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 21, 2023 15:23 Inactive
@HendrikThePendric HendrikThePendric self-assigned this Jun 21, 2023
@dhis2-bot dhis2-bot temporarily deployed to netlify July 3, 2023 14:37 Inactive
@HendrikThePendric HendrikThePendric marked this pull request as draft July 4, 2023 16:03
@HendrikThePendric
Copy link
Contributor Author

As agreed with @janhenrikoverland I returned this PR to draft and added the "on hold" label. We won't be doing any releases until august.

@janhenrikoverland janhenrikoverland changed the title feat: toolbar UI update feat: toolbar UI update (DHIS2-15167) Aug 9, 2023
@janhenrikoverland janhenrikoverland marked this pull request as ready for review August 9, 2023 08:57
@dhis2-bot dhis2-bot temporarily deployed to netlify August 9, 2023 08:59 Inactive
@janhenrikoverland janhenrikoverland merged commit 6c54186 into master Aug 9, 2023
11 checks passed
@janhenrikoverland janhenrikoverland deleted the DHIS2-15167-toolbar-update branch August 9, 2023 09:34
dhis2-bot added a commit that referenced this pull request Aug 9, 2023
# [100.9.0](v100.8.2...v100.9.0) (2023-08-09)

### Features

* toolbar UI update (DHIS2-15167) ([#368](#368)) ([6c54186](6c54186))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 100.9.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants